Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Hyperspace mantis #515

Draft
wants to merge 23 commits into
base: hyperspace-solana
Choose a base branch
from
Draft

WIP: Hyperspace mantis #515

wants to merge 23 commits into from

Conversation

dhruvja
Copy link
Collaborator

@dhruvja dhruvja commented Sep 6, 2024

No description provided.

@dhruvja dhruvja requested review from mina86 and vmarkushin and removed request for mina86 September 6, 2024 08:13
@dhruvja dhruvja marked this pull request as draft September 6, 2024 08:14
dhruvja and others added 19 commits September 6, 2024 14:11
For cf-solana, replace dependencies to refer to git repository rather
than using path pointing outside of the repository and fix paths for
DecodeError and BadMessage types.

For ics10-grandpa-cw remove unnecessary as_bytes method calls which
were there presumably because SUBSTITUTE_PREFIX and SUBJECT_PREFIX
consts were different type at some point.
This addresses the following error present whet `std` feature is
enabled:

    error: couldn't read .../debug/build/parachain-runtime-7decbbfb16bf29dd/out/wasm_binary.rs: No such file or directory (os error 2)
      --> utils/parachain-node/runtime/src/lib.rs:21:1
       |
    21 | include!(concat!(env!("OUT_DIR"), "/wasm_binary.rs"));
       | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Replace Anchor dependency with a fork which diverges less from 0.29.
The previous one had 30 commits on top of v0.29.0 tag (with 29 pulled
from Anchor master).  The new one is v0.29.0 with just a single commit
that we need on top of it.
Previous change adding witness account to solana-ibc calls was
incomplete as it didn’t add the dependency.  I missed that the code
wasn’t actually compiling due to many warnings polluting the build.
This commit fixes all that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants